Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lint Generator: No longer fix violations after invocation #1174

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

stevepolitodesign
Copy link
Contributor

Supports #1152

We found that we needed to run this generator last in suspenders:install:web to ensure any potential violation was resolved.

Instead, we'll just make that the responsibility of suspenders:install:web.

stevepolitodesign added a commit that referenced this pull request Mar 25, 2024
For some reason #1175 passed, but CI for #1174 is failing.
stevepolitodesign added a commit that referenced this pull request Mar 25, 2024
For some reason #1175 passed, but CI for #1174 is failing.
Supports #1152

We found that we needed to run this generator last in
`suspenders:install:web` to ensure any potential violation was resolved.

Instead, we'll just make that the responsibility of
`suspenders:install:web`.
@stevepolitodesign stevepolitodesign merged commit e7926f1 into suspenders-3-0-0 Mar 25, 2024
2 checks passed
@stevepolitodesign stevepolitodesign deleted the suspenders-3-0-0-lint-follow-up branch March 25, 2024 13:34
stevepolitodesign added a commit that referenced this pull request May 10, 2024
For some reason #1175 passed, but CI for #1174 is failing.
stevepolitodesign added a commit that referenced this pull request May 10, 2024
Supports #1152

We found that we needed to run this generator last in
`suspenders:install:web` to ensure any potential violation was resolved.

Instead, we'll just make that the responsibility of
`suspenders:install:web`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant